Support embedded objects #829
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #829 +/- ##
============================================
+ Coverage 0 94.31% +94.31%
============================================
Files 0 10 +10
Lines 0 211 +211
============================================
+ Hits 0 199 +199
- Misses 0 12 +12
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
example/lib/type_converter.dart
Outdated
| @@ -24,3 +24,15 @@ class TaskTypeConverter extends TypeConverter<TaskType?, String?> { | |||
| return value?.name; | |||
| } | |||
| } | |||
|
|
|||
| class TaskStatusConverter extends TypeConverter<TaskStatus?, String?> { | |||
There was a problem hiding this comment.
This one is not needed, because TaskStatus is an enum
example/lib/timestamp.dart
Outdated
| int get hashCode => createdAt.hashCode ^ updatedAt.hashCode; | ||
|
|
||
| @override | ||
| String toString() { |
There was a problem hiding this comment.
I don't really like using the toString function to for user-facing strings. I think it would be better to keep toString for technical things and introduce a new function, something like: format() to get a pretty user-facing String
|
This is already a good thing, but I noticed that it does not work if the embeded attribute is nullable. Can you add support for that? |
This pull request introduces support for embedded objects in the Floor library. Initially, I referred to pull request #565 but due to the extensive outdated changes, I decided to start fresh to avoid complex merge conflicts.